package net.weitalk.idgenerator.v1;

import java.net.InetAddress;
import java.net.UnknownHostException;
import java.util.Random;

import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

/*
RandomIdGenerator 设计成了实现类而非接口，调用者直接依赖实现而非接口，违反基于接口而非实现编程的设计思想。
如果项目中需要同时存在两种 ID 生成算法，也就是要同时存在两个 RandomIdGenerator 实现类。比如，我们需要将这个框架给更多的系统来使用。
系统在使用的时候，可以灵活地选择它需要的生成算法。这个时候，我们就需要将 RandomIdGenerator 定义为接口，并且为不同的生成算法定义不同的实现类。
 */
public class IdGenerator {
    private static final Logger logger = LoggerFactory.getLogger(IdGenerator.class);

    //把 RandomIdGenerator 的 generate() 函数定义为静态函数，会影响使用该函数的代码的可测试性。
    //同时，generate() 函数的代码实现依赖运行环境（本机名）、时间函数、随机函数，所以 generate() 函数本身的可测试性也不好，
    //需要做比较大的重构。
    public static String generate() {
        String id = "";
        try {
            String hostName = InetAddress.getLocalHost().getHostName();
            String[] tokens = hostName.split("\\.");
            if (tokens.length > 0) {
                hostName = tokens[tokens.length - 1];
            }
            char[] randomChars = new char[8];
            int count = 0;
            Random random = new Random();
            while (count < 8) {
                int randomAscii = random.nextInt(122);
                if (randomAscii >= 48 && randomAscii <= 57) {
                    randomChars[count] = (char) ('0' + (randomAscii - 48));
                    count++;
                } else if (randomAscii >= 65 && randomAscii <= 90) {
                    randomChars[count] = (char) ('A' + (randomAscii - 65));
                    count++;
                }  else if (randomAscii >= 97 && randomAscii <= 122) {
                    randomChars[count] = (char) ('a' + (randomAscii - 97));
                    count++;
                }
            }
            id = String.format("%s-%d-%s", hostName,
                System.currentTimeMillis(), new String(randomChars));
        } catch (UnknownHostException e) {
            logger.warn("Failed to get the host name.", e);
        }
        return id;
    }
}
//虽然 RandomIdGenerator 只包含一个函数，并且代码行数也不多，但代码的可读性并不好。
//特别是随机字符串生成的那部分代码，一方面，代码完全没有注释，生成算法比较难读懂，
//另一方面，代码里有很多魔法数，严重影响代码的可读性。在重构的时候，我们需要重点提高这部分代码的可读性。
